Skip to content

Publish SNAPSHOT artifacts during PR builder#876

Merged
thibauult merged 13 commits into
mainfrom
cicd/publish-snapshots-in-pr-builder
Jun 16, 2026
Merged

Publish SNAPSHOT artifacts during PR builder#876
thibauult merged 13 commits into
mainfrom
cicd/publish-snapshots-in-pr-builder

Conversation

@thibauult

@thibauult thibauult commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds a Publish SNAPSHOT artifacts step to the PR builder workflow
  • The step only runs on pull_request events (skipped on pushes to main/*-rc)
  • Uses publishToSonatype which automatically targets the Sonatype snapshot repository for -SNAPSHOT versions
  • No GPG signing required — already gated on isReleaseVersion in bdk.java-publish-conventions.gradle

Prerequisites

MAVEN_USERNAME and MAVEN_PASSWORD repository secrets must be set (already used by release.yml).

Test plan

  • Open a PR and verify the Publish SNAPSHOT artifacts step runs and succeeds
  • Verify artifacts are published to https://central.sonatype.com/repository/maven-snapshots/
  • Verify the step is skipped on a direct push to main

thibauult added 13 commits June 10, 2026 10:10
…tion@v4

The old gradle/wrapper-validation-action is deprecated and was causing CI failures.
CVE-2026-42582 (Netty 4.1.134.Final) and CVE-2026-41840/41841/41842/41843/41850/41851
(Spring Framework 6.2.18 via spring-boot-dependencies:3.5.14) have no fixed release yet.
Suppressed temporarily to unblock CI; a follow-up PR will bump the dependencies once
fixed versions are available.
3.5.15 ships Netty 4.1.135.Final (fixes CVE-2026-42582 and 18 additional Netty CVEs)
and Spring Framework 6.2.19 (fixes CVE-2026-41840 through CVE-2026-41851). Also ships
Tomcat 10.1.55 so the explicit overrides for Netty and Tomcat are no longer needed.

Reverts the CVE suppressions added in the previous commit.
Netty 4.1.135.Final is the last 4.1.x release and Spring Boot 3.5.15
is the last 3.5.x release — no patched version is available upstream.
…ions

Move secrets to job-level env vars and use env context in the conditional
expression to check whether Sonatype credentials are available before
publishing snapshot artifacts.
Sets the artifact version to PR-{number}-SNAPSHOT on PR builds so each PR
publishes an independently addressable SNAPSHOT. Reports the published version
in the GitHub Actions step summary.
@github-actions

Copy link
Copy Markdown

SNAPSHOT published: PR-876-SNAPSHOT
Repository: https://central.sonatype.com/repository/maven-snapshots/

@symphony-enrico symphony-enrico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sebastientosello

Copy link
Copy Markdown

greaaattttt 🚀

@thibauult thibauult merged commit aead2eb into main Jun 16, 2026
5 checks passed
@thibauult thibauult deleted the cicd/publish-snapshots-in-pr-builder branch June 16, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants